From: Florian Müllner <fmuellner@gnome.org>
Date: Fri, 25 Apr 2025 12:48:23 +0000 (+0200)
Subject: workspace: Simplify hasMaximizedWindows()
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~124^2^2~99
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=ec7ccabe282921945fca3d164f3b35f1fa05950a;p=gnome-shell.git

workspace: Simplify hasMaximizedWindows()

Since mutter exposes `is_maximized()` now, use that instead
of checking for both horizontal and vertical maximization.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3704>
---

diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index adbecb28b..25c39aa5d 100644
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -1273,14 +1273,11 @@ class Workspace extends St.Widget {
 
     // check for maximized windows on the workspace
     hasMaximizedWindows() {
-        for (let i = 0; i < this._windows.length; i++) {
-            let metaWindow = this._windows[i].metaWindow;
-            if (metaWindow.showing_on_its_workspace() &&
-                metaWindow.maximized_horizontally &&
-                metaWindow.maximized_vertically)
-                return true;
-        }
-        return false;
+        return this._windows.some(w => {
+            const {metaWindow} = w;
+            return metaWindow.showing_on_its_workspace() &&
+                   metaWindow.is_maximized();
+        });
     }
 
     _clearSkipTaskbarSignals() {